Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transfer GT++ ASM fixes to mixins in hodgepodge #423

Merged
merged 3 commits into from
Sep 21, 2024
Merged

Conversation

Alexdoru
Copy link
Member

@Alexdoru Alexdoru commented Sep 21, 2024

The keyboard fix in gt++ was 600 lines of broken ASM combined with reflection when it could be a 5 lines mixin

Transfered the thaumcraft ItemWispEssence fix

Transfered the cofh npe fix

See : GTNewHorizons/GT5-Unofficial#3238

@Alexdoru Alexdoru marked this pull request as draft September 21, 2024 00:54
@Alexdoru Alexdoru changed the title fix array out of bounds caused by the GameSettings.getKeyDisplayString method Transfer GT++ ASM fixes to mixins in hodgepodge Sep 21, 2024
@Alexdoru Alexdoru marked this pull request as ready for review September 21, 2024 17:08
@Alexdoru Alexdoru requested a review from a team September 21, 2024 17:08
@mitchej123 mitchej123 merged commit 610820f into master Sep 21, 2024
1 check passed
@mitchej123 mitchej123 deleted the keyboard-fix branch September 21, 2024 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants